Skip to content

fix: place-order form below the orders list (stops the filter tab-jump) - #2

Merged
DanMat merged 1 commit into
mainfrom
fix/workbench-layout
Aug 30, 2026
Merged

DanMat merged 1 commit into
mainfrom
fix/workbench-layout

Conversation

@DanMat

@DanMat DanMat commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Live-demo feedback (Phase 1 follow-up): with the place-order form above the status filter, changing a filter tab — a full page load that resets scroll to the top — bounced the operator back up to the form each time.

  • Lead with the status filter + orders list; move Place an order to the bottom, so filtering lands on the list, not the form.
  • Give the place form top spacing from the table.

Pure presentation; 22 tests green, PHPStan + php-cs-fixer clean.

🤖 Generated with Claude Code

… jump)

Feedback from the live demo: because the place-order form sat above the status
filter, changing a filter tab (a page load that resets scroll to the top)
bounced the operator up to the form every time. Lead with the filter + orders
list and move the place form to the bottom, so filtering lands on the list.
Give the form top spacing from the table.

Pure presentation; 22 tests still green.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@DanMat
DanMat merged commit 65cac7e into main Aug 30, 2026
2 checks passed
@DanMat
DanMat deleted the fix/workbench-layout branch August 30, 2026 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant